merged master

Change-Id: I6cf08c09c7d9b38ecce0b2bbed61431939edd7d4

Merge "PHP test suite for CSSMin"

PHP test suite for CSSMin

Change-Id: Ic9195614acfd616ccdff57cfc666aa4dfa71fb96

Merge "(bug 37458) permission errors running BaseDumpTest on Windows"

(bug 37458) permission errors running BaseDumpTest on Windows

Change-Id: I58264c5f516f8dcb124d753b43a80b0875b0e974

Merge "(bug 35727) mw.Api ajax() should put token parameter last."

Merge "(bug 37301) add sizediff to list=usercontribs"

Merge "Can't use return value of void functions"

Merge "Bump release notes to say jQuery UI 1.8.21 (seems I forgot to do this in the commits...)"

Merge "Kill RELEASE-NOTES-1.19 from core. Not needed anymore"

(bug 35727) mw.Api ajax() should put token parameter last.

Patch from Russell Blau.

As recommended in [[mw:API:Edit#Token]], the "token" parameter in
a request should always be passed at the end of the query string.
This is a safety measure in case transmission of the HTML request to
the server is interrupted; then the server will not process the
incomplete request because there will be no (complete) token.
(Conversely, if the "text=" parameter were last, the server would have
no way of knowing whether the complete text had been received.)
Presumably the same thing is necessary for action=email, since the
request may include text of arbitrary length.

Change-Id: Id267dd628eb93eb06191c55eb386b1893f499554

Merge "Localisation updates from translatewiki.net."

Merge "Put skin classes in the Autoloader"

Localisation updates from translatewiki.net.

Change-Id: Ic387b6f35cf3fb1931bb3d8c52efee17b91cf1be

support symbolic name for content models

Change-Id: I4eaed3db7fb10069f73d84514abd2f6957019153

Merge "undo misguided changes to phpunit" into Wikidata

undo misguided changes to phpunit

Change-Id: I14f72e3ea6885cd5e0fbdfe6aca42d39076274ec

Fix indenting errors introduced in Wikidata branch

Change-Id: I487a0966a26359d4a6769331a6e2ee356eeb41db

Fix comments added in change 10088

I found my comments in change 10088 is inadequate because we don't
have any known squid URLs in Title::getSquidURLs() and we may never
want to have... There's no hook there so extensions are unable to
add any squid URLs for them (maybe we want one someday? but new
extensions shouldn't be using the UnknownAction hook).

Change-Id: I44ba376f92776ec6bb32f91beb06f11ebd086fb4

Merge "For consistency of the interface, don't do expensive checks on page view."

SkinTemplate: Minor clean up

* Better variable names ($wgContLang is object, $userLang was a string
  and $lang the object. Made $userLang the object and $userLangCode
  the string.

* Removed trailing whitespace.

* Spacing (not all just where it fit better in the context, some
  isset($) to isset( $ ). and a few `if()` to `if ()` since most
  in this file are like that.

* Some redundant magic quote usage to single quotes.

* Weird structure in getPersonalTools():
  Instead of accessing the same deep array property over and over
  again to add something to the newly create array (e.g.
  $foo[] = array(); and then $foo[0] = $bar;) simply creating it
  as a literal at once.

Change-Id: I861ae1d558f324f61c79089dcd3bff52dd27e0aa

Localisation updates from translatewiki.net.

Change-Id: Ica921689fbb2051272010d77198e5932421fa9e9

Merge "ensure unique revision id in backupTextPassTest"

Merge "fixed bug 37458: permission errors running BaseDumpTest on Windows" into Wikidata

fixed bug 37458: permission errors running BaseDumpTest on Windows

patch set 2: removed trailing whitespace;

Change-Id: I58264c5f516f8dcb124d753b43a80b0875b0e974

made DifferenceEngine::getRevisionHeader() protected, so we can override it.

Change-Id: Ic429cd0c039c0eb0487e149f9bb79c469b3635b3

Kill RELEASE-NOTES-1.19 from core. Not needed anymore

Change-Id: I41df1c6201a3ad22e2d1824102eecc02be719657

Bump release notes to say jQuery UI 1.8.21 (seems I forgot to do this in the commits...)

Change-Id: Ic1c285582b9c92a8510ac67cf26f5787ec3bb59b

Merge "Update and minor clean up of release-notes"

Merge "Remove duplication of Langauge digitTransformTable in javascript."

Remove duplication of Langauge digitTransformTable in javascript.

Use the new ResourceLoaderLanguageDataModule for passing the digit
transform table and digit separators (comma or dot) from the PHP
classes. And remove the duplicative javascript files so we no longer
have to maintain changes to MessagesXYZ.php and languages/xyz.js.

Change-Id: Icf63ebf927f993fe04518c4afe3a0b6959f5b1bd

Update and minor clean up of release-notes

* also for 5b5cdec7e56dd12a45f915766cdc9eaf62174cd6.
  (QUnit update to v1.8.0).

  We've updated QUnit about 5 times to upstream release
  during 1.20 development, when is our release coming :)

Change-Id: Iac101f29669f13b0d8a6865f7f1849062eae7537

Merge "Update jquery.qunit from upstream v1.7.0 to v1.8.0"

Update jquery.qunit from upstream v1.7.0 to v1.8.0

* Upstream https://github.com/jquery/qunit.git
 - tag: v1.8.0
 - tree: https://github.com/jquery/qunit/tree/v1.8.0
 - commit: https://github.com/jquery/qunit/commit/8a7fc96381b14792ff1c380ffa6682396658a4c2

* Misc:
 - Updated existing release-note entry for QUnit update
 - This release introduces the query parameter "module=", which,
   contrary to "filter=" reduces the tests to be run to only tests
   within the module by that exact name ("filter" does a string search
   so filter=foo also matched "module: bar, test: does bafoola quux?".

   We should update our TestSwarm configuration to use this parameter
   instead.

Change-Id: Iee3f40ca5d50694038e759268c07c2694c98bbc6

Localisation updates from translatewiki.net.

Change-Id: I5a2edc647e05551a87cc76a1b397030eb6c42b16

Merge "Update and fix AutoLoader definitions."

Merge "Improve documentation of maintenance scripts."

Merge "Wrap auto-numbering for section heading in a classed span (bug 33450)"

Merge "(bug 21660) "Pipe trick" full width commas (with test!)"

Improve documentation of maintenance scripts.

Change-Id: I328a3d42a09f2b082ebc153c2f399793e23942d7

Update and fix AutoLoader definitions.

All these changes were found by running maintenance/checkAutoLoader.php.

Change-Id: Iffef080c8b0156af1fc6d4651c07e0e31e9c1674

Merge "Double the size of the target input field of Special:Contributions"

Can't use return value of void functions

Change-Id: Idce6e31f3f0eff0a0045c1cc62def3f5d79b9d48

(bug 37301) add sizediff to list=usercontribs

Moving function getParentLength from SpecialContributions to Revision
and use it in the list module. The size of the parent is selected by a
batch, like the special page.

Change-Id: I6e388e75cd765f2a918b0c192477d87347e96bcd

ensure unique revision id in backupTextPassTest

Idb82fe620c3395acaa8f051b4d4d0d770cd33d44 adds a unique contraints on
the revision id to export.xsd, which does not apply here
creating unique contraints in the same way than the pageids by adding an
offset

Change-Id: Ia19f1011a5c0ba8d21cc01a9be2e4d2ed300a80a

Merge "(bug 33222) Add parentid to revision in export xml"

Merge "Revert "Deprecate no longer used Skin::getCommonStylePath.""

Revert "Deprecate no longer used Skin::getCommonStylePath."

This reverts commit 79dcf7686d857e1dcbc3f1127a106d847de10d9e

Localisation updates from translatewiki.net.

Change-Id: I19b9e057f8ed555178480d04b12aa17c87bc97b3

Merge "[FileBackend] Process cache fixes and cleanups."

Merge "Revert "Deprecate no longer used Skin::getCommonStylePath.""

Merge "Fixes to LogEventsList."

Use canonical class name ApiBase also in ApiQueryLangLinks

Change-Id: I11c7b0540078e6934c8d9d8fa21891a87bfebd21

Use canonical class name for ApiBase
 s/Apibase::PROP_NULLABLE/ApiBase::PROP_NULLABLE/

Change-Id: I222df124f947face5970bab810f497c43000dbf2

Fixes to LogEventsList.

* Removed useless variables
* Use Linker::linkKnown() instead of Linker::link() where possible

Change-Id: I7cb668ef929cee159946a46cbcb07464a0913f81

Merge "Added ORMIterator interface which can be used for type hinting (in particular when passing ORMResult objects)"

Merge "Fixup some wrong documentation"

Fixup some wrong documentation

Don't try and return the result of void functions

Change-Id: I7a4db738b3e3052aebcf43742d31753f8cc1f47a

Minor additions to documentation stuff

Change-Id: I79417ce790c22a3f6ad1d24153da898901d1142f

Merge "(bug 34933) Create "Check: [All] [None]" buttons with JavaScript"

Merge "(bug 15853) Wrong entries in history feed"

Merge "Added some extra tests for ORMRow class"

Merge "Do proper conversion table detection."

Merge "(bug 37452) Added class="mw-redirect" to redirect links in Special:PrefixIndex"

Added a hook for formatting autocomments

Removed some spurious whitespace

Changed how wfRunHooks are used, and also the test for default formatting

Change-Id: I4c3fc1080f83166d7b89aeb9a1487e173e9ccb65

Put skin classes in the Autoloader

Also remove require_once() calls from skin files.

This fixes a bug introduced in 0ec77e2b31 where MySkin would throw a
fatal error because it couldn't find the MonoBookTemplate class.

Change-Id: I63bbe1d25e8cb70ee4baf99ec6e685444b663c1f

Revert "Deprecate no longer used Skin::getCommonStylePath."

This reverts commit 79dcf7686d857e1dcbc3f1127a106d847de10d9e.

getCommonStylePath and getSkinStylePath are a pair of methods for use in 3rd party skins (NOT core).
They cut down on the boilerplate for including skin urls into skins for things like <img> tags.
Without these methods skin authors will be forced to revert to unintuitive boilerplate to add style paths and style versions into their code.
Most won't even bother to include the style version.
And ResourceLoader is a tangent to the whole topic because no matter what the advantages to CSS are developers writing custom skins for a single site from scratch will not care and they will choose CSS or <img> based on situations on hand in their own markup that make one or the other harder to handle.
This addition was part of the effort to fix the longstanding issue that our our skin system does not properly account for 3rd party skin developers.

Change-Id: I98f4fd83c93aba7681306d0a1edb58f798057543

Merge "Check validity of fetched titles and show a descriptive messages when an invalid title is encountered."

Merge "Deprecate no longer used Skin::getCommonStylePath."

Merge "(bug 27283) SqlBagOStuff breaks PostgreSQL txns"

(bug 27283) SqlBagOStuff breaks PostgreSQL txns

* SqlBagOStuff should not turn off automatic transaction
    control just like that. Keep previous
    connection for PostgreSQL and SQLite

* 94633ff448e9253c5503124df6748be1fc26b8e2 introduced
    rollback on error to restore sanity
    for the connection. Don't do this
    if you are in the middle of INSERT IGNORE
    and we have a savepoint open.

    Making sure query syntax errors don't
    get silenced by our "wonderful" implementation
    of INSERT IGNORE is bug 35572.

Change-Id: I841b03895e1189c47307fefb1516c4c7c4102e25

Merge "avoid Title::exist/getArticleId in ApiQueryInfo::extractPageInfo"

Merge "fix backup unit tests"

Localisation updates from translatewiki.net.

Change-Id: I1c650dacc81e725610740e6ec773bbcbd8bcbec8

Allowing the change password form to be extended

Change-Id: Id676b1ec4b43140790b18a87bc8466f10c957577

use 'contentformat' instead of 'textformat' in result

Change-Id: If2d392329bb1618dbacf489f96b53115281b8e97

Merge "Correct case for the "MediaWiki:Revdelete-reason-dropdown" link in Special:RevisionDelete."

Merge "Use local context to get messages"

Merge "Use local context instead of hidden dependencies on global variables."

Added missing GPLv2 headers in some places.

Also made file/class documentation more consistent.

Change-Id: I8ee0861bfc2e9ff6511ad61175f1f1273739816c

Merge branch 'master' of ssh://gerrit.wikimedia.org:29418/mediawiki/core into Wikidata

Use class-static var for storing ContentHandler singletons.

$wgContentHandlers will be used for configuration only, and
is no longer changed during operation.

It's no longer possible to supply instances directly in $wgContentHandlers,
it only takes class names now.

Change-Id: Ifafb08d8638eff0d6965cd92fadbd9071c74de37

Clean up comments and whitespace

* Break most comment lines at 80 columns
* Break code lines at 100 columns or less
* Fix capitalisation in comments
* Fix several instances of reversed name and type in @param, Doxygen cares
* Replaced @param with @return in several places
* Use a Doxygen format list in doc comments where appropriate
* Reduced comment duplication in subclasses, for maintainability
* General copyediting of comments
* Fixed a couple of other minor code style issues

Change-Id: I3a6f9a6870a9b859c1bd4251202acba607fbf2b7

Use message 'brackets' for brackets

Change-Id: I92419cb6d86e313b611b1f7132a278f7f8f97d5f

lint mediawiki.Uri.js

* remove dangling _
* var hosting
* whitespace conventions

Change-Id: I7c78059f640dfdf52020018425026cb714bcdbfa

Added @group database to ContentHandlerTest.

Apparently, the parser needs database access to handle local links using
a LinkHolderArray. This fails in the "databaseless" test run performed by jenkins.

Change-Id: I7f242abea0c25c07638b4bca4a6732289a3a7e91

Merge "Fixed pecl memcached client in persistent mode."

Fixed pecl memcached client in persistent mode.

* When using a persistend ID in the constructor, the object is effectively shared among
  the callers using that ID. If we always use __CLASS__, then this breaks when different
  BagOStuffs may be used. Instead, a hash of the settings is now used.
  See http://www.php.net/manual/en/memcached.construct.php.
* Don't keep adding servers to the pool, as the object is shared among callers. Once one thread
  initializes the object, it will already be initialized by the next thread. Calling addSever()
  again will cause an increasing number of duplicate servers to exist in the data structure, and
  thus an increasing number of socket connections over time.
* Also, use addServers() instead of multiple addServer() calls per
  http://www.php.net/manual/en/memcached.addserver.php.

Change-Id: I0e7510320cb79d9f152e8958ddd50400fa9da37f

Fix for 15e5193a19, broke in debug mode

We shouldn't be combining styles in debug mode anyways.

Change-Id: I8f0503e665df870e0940314e10185b94624869b9

Fix media type handling for CSS that's not loaded with only=styles

We were only doing server-side @media wrapping in only=styles mode but
not in regular mode. Meanwhile, the client-side loader had dropped all
support for media types under the assumption that this was all handled
server side, which caused media type handling to break for modules
loaded the normal way. We didn't notice this before because the only
core module that uses this is skins.vector, which is loaded through
only=styles.

Change-Id: I8228cc0317328440256b45b166eef1b57dbf4602

Merge "Localisation updates from translatewiki.net."

Allow cleanupSpam.php optionally delete offending pages

Change-Id: I69387df1b3d404a6d00d8073e58d7556c7ab9298

Localisation updates from translatewiki.net.

Change-Id: I66636cc9e42eb82cdf12b4974a48ba9264fae4c2

Bug 24985 use $wgTmpDirectory when available

We had two way to get a temporary directory:

 - $wgTempDirectory: more or less stable accross sessions
 - wfTempDir(): set through environnement variable and could potentially
   vary from a session to another one thanks to tempnam()

This patch makes wfTempDir() to always use the global $wgTempDirectory
first when it is available. Thus explicitly overriding tempnam() or any
environnement variable such as TMP or TEMP.

Hence, people who don't have access to a system wide directory
specificed by their environnement (such as /tmp) can specify an
alternative straight from the MediaWiki configuration.

The patch remove references to $wgTmpDirectory and replace them with
calls to wfTempDir().  Make wfTempDir() use $wgTmpDirectory first.

The default setting of $wgTmpDirectory was removed in favor of having it
initialized through Setup.php by calling wfTempDir.

Note: this may also address Bug 36475 - Generating thumbnails does not
work when there is no access to /tmp

Change-Id: Ifdc79e9c5d95f978025b237a5eeb95fd75092f46

Merge "(bug 37090) Remove Spanish gender aliases."

(bug 37090) Remove Spanish gender aliases.

Removed for all languages that have Spanish as fallback and have their own
namespace names for NS_USER that differs from Spanish.

Left untouched: arn, ay.

Also removed trailing whitespace from file.

Change-Id: I6f2d7029174e9506f8e245f765cbc0db67a63099

Merge "(bug 36938) XSS in uselang parameter"

Merge "(bug 34988) Namespaces names in Silesian (szl)."

(bug 34988) Namespaces names in Silesian (szl).

Change-Id: I5b2b2ed4b3e31af8d7f03e8c00a6c00f5984cb24

fixed LinksUpdateTest: must use transaction.

Change-Id: I9d80a9946a01bfa9245082aa29f1624c505d4201